-
Notifications
You must be signed in to change notification settings - Fork 1.4k
Enable automatic code signing in Xcode #2347
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: dev
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In all other places in this file, where the syntax
DEVELOPMENT_TEAM = "$(LOOP_DEVELOPMENT_TEAM)";
is used, you see this syntax:
CODE_SIGN_IDENTITY = "$(LOOP_CODE_SIGN_IDENTITY_DEBUG)";
CODE_SIGN_STYLE = "$(LOOP_CODE_SIGN_STYLE)";
DEVELOPMENT_TEAM = "$(LOOP_DEVELOPMENT_TEAM)";
--
CODE_SIGN_IDENTITY = "$(LOOP_CODE_SIGN_IDENTITY_RELEASE)";
CODE_SIGN_STYLE = "$(LOOP_CODE_SIGN_STYLE)";
DEVELOPMENT_TEAM = "$(LOOP_DEVELOPMENT_TEAM)";
Where those environment variables are configured in Loop.xcconfig
// Code signing and provisioning [DEFAULT]
LOOP_CODE_SIGN_IDENTITY_DEBUG = Apple Development
LOOP_CODE_SIGN_IDENTITY_RELEASE = Apple Development
LOOP_CODE_SIGN_STYLE = Automatic
I expect @ps2 will request that you follow that model, so I am requesting changes. If he says it is not important, then I will withdraw my request for changes.
TestThe functional behavior is independent of whether "Apple Development" is entered directly for the CODE_SIGN_IDENTITY, so these tests are valid and successful. ✅ Mac-Xcode build: no issues building with this change. |
Having a write up for how to use Xcode Cloud will be excellent. Thank you for doing this. |
@marionbarker thank you for testing! Yes, with this change there can be some variables removed from the xcconfig in the LoopWorkspace repo including: LOOP_CODE_SIGN_IDENTITY_DEBUG Let’s definitely let @ps2 chime in! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks! LGMT. Removing those other unused settings would be nice too.
I want to be clear which settings should be removed. Loop.xcconfig has these 3 lines:
I was suggesting that those variables be used in @Camji55 refers to those being in the LoopWorkspace: |
This change allows building and distributing with Xcode Cloud which I have been doing on my personal branch for some time now. The intent is that existing workflows for browser builds with GitHub Actions and all Continuous Integration actions should still work as-is since Fastlane should override these code-signing settings. This will need some testing before it gets adopted but having the option to build and deploy within Xcode itself would be beneficial for users who'd like to take advantage of it especially if other options still remain available.
I'm also working on a write up on how to setup Xcode Cloud as an alternative to GitHub Actions for browser builds.
Please let me know what I can do to help verify all Fastlane and GitHub Actions remain working!